-
Notifications
You must be signed in to change notification settings - Fork 103
feat: Uv take 3, no pyproject.toml and with pinned setuptools and wheel #5369
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Uv take 3, no pyproject.toml and with pinned setuptools and wheel #5369
Conversation
jjbayer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's try it! I did a few smoke tests, like running python setup.py sdist and committing badly formatted python, that all seems to work 👍 .
I can cut a release of the python library right after merging so there are no surprises at a later point.
| # TODO: reactivate when fixed | ||
| # - id: check-github-workflows |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this TODO for this PR, or for later?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\shrug
…t-toml-with-pinned-setuptools
This brings back my uv pr without pyproject.toml and with pinned setuptools and wheel.
The original error doesn't refer to which artifact failed to upload and in fact it is referring to the two macos wheels in the release artifact here. These wheels have metadata 2.1 but also Dynamic fields which are a 2.2+ thing.
I determined this was due to pinning wheel 0.42.0 in my uv pr which produces bad macos wheels. 0.45.1 works. This worked before because in CI we were installing unpinned (latest) setuptools+wheel.
I wrote a small script that reproduced twine's upload fail here (wheel 0.42.0) so that we can error out before actions/upload-artifact and before any release artifacts are published to github.
#skip-changelog